View Issue Details

IDProjectCategoryView StatusLast Update
0001626T99X171.00 SKB EaglexTS certificationpublic2023-02-07 16:25
Reporter(SW) Jason LingAssigned To(SW) Sam Yang Due Date
PrioritynormalSeveritys4-minorReproducibilityalways
Status closedResolutionfixed 
Summary0001626: [Smart3][ATV12] Press volume key cause SetupWizard to crash
DescriptionHi JunGyu,

Please check the reproduce step as below,

1. Do Google factory Reset
2. Press the volume key on any page after the 2nd page of Setupwizard
3. SetupWizard will crash and back to 1st page.

it will Impact SmokeTest (setup-flow-robustness)

Please check it with SPTEK. I think there might be something wrong with the key handling function.

Thanks,
Jason
TagsNo tags attached.
Attach Tags

Users monitoring this issue

Activities

(ALTech) JunGyu Kim

2023-01-31 09:02

developer   ~0011896

Hi Jason,

I reported this issue to SPTEK. (https://jira.skbroadband.com/browse/FSTB12-72)
I will let you know if any update.

Thank you.
Kim

(ALTech) JunGyu Kim

2023-01-31 12:40

developer   ~0011911

Hi Jason,

SPTEK said that, need to be check sepolicy first.
=======================================================================================================================================
01-30 21:18:21.515 3126 3126 E SELinux : avc: denied { find } for pid=5249 uid=10052 name=media_session scontext=u:r:setupwraith_app:s0:c512,c768 tcontext=u:object_r:media_session_service:s0 tclass=service_manager permissive=0
01-30 21:18:21.517 3715 3899 D WindowManager: keycode : 24 - processed. by skb function key
01-30 21:18:21.519 5887 5887 I BtvService_6.2.93 PeripheralService: APK BTF|onReceive|54| LEDControl:onReceive : com.skb.intent.KEY_LED_BLINK
01-30 21:18:21.519 3126 3126 E SELinux : avc: denied { find } for pid=5249 uid=10052 name=media_communication scontext=u:r:setupwraith_app:s0:c512,c768
=======================================================================================================================================

Could you check it?
Thank you.
Kim

(SW) Jason Ling

2023-02-01 11:56

developer   ~0011939

Hi JunGyu,

We use the pure Amlogic SDK and + GTVS (same as the 537 branch)
we don't have this issue.

I think SPTEK modified framework then cause Selinux issue in SetupWizard.
The same SetupWizard works fine with pure Amlogic SDK.

Thanks,
Jason

(ALTech) JunGyu Kim

2023-02-01 14:14

developer   ~0011946

Hi Jason,

The reason of crash is SetupWraith apk does not have media_session_service permission.
(SPTEK is fixed that Audioservice is called directly by Phonewindowmanger. It has been fixed due to the problem when long-press volume key.)
Is it possible to add media_session_service permission in SetupWraith apk?

Thank you.
Kim

(SW) Jason Ling

2023-02-01 15:12

developer   ~0011956

Hi JunGyu,

I can not fully understand your request.
>>It has been fixed due to the problem when long-press volume key
Did you mean this issue has been fixed by SPTEK?

>>Is it possible to add media_session_service permission in SetupWraith apk?
We haven't modified any sepolicy for Google GTVS packages, we don't plan do it.

Thanks,
Jason

(ALTech) Sangmin Choi

2023-02-02 13:23

developer   ~0011981

Hi, Jason,

I think SPTEK modified framework then cause Selinux issue in SetupWizard.
The same SetupWizard works fine with pure Amlogic SDK.
-> You're right, I also agree that SPTEK should add sepolicy rule to fix this issue, because they modified framework.
But, other vendor already added sepolicy rule and they already checked this issue has fixed.
So, SPTEK asked manufactures to add sepolicy rule.
As I mentioned first, SPTEK should add sepolicy rule, but it's difficult sending this job to SPTEK, because other vendor already supported it.

Could you please kindly consider to add sepolicy again?
FYI, I heard that this issue(SetupWizard will crash and back to 1st page) has fixed after add sepolicy rule.

Thank you.
Sangmin Choi.

(SW) Jason Ling

2023-02-02 15:30

developer   ~0011983

Hi Sangmin,

Sam will check it.

Thanks,
Jason

(SW) Jason Ling

2023-02-07 16:24

developer   ~0012045

Hi Sangmin,

Sam fix it

commit d33c955b4a09917aad4634d46a7dd16a03851241
Refs: [HEAD], {m/BFX-AT100_5.3.7}, {skb/BFX-AT100_5.3.7}, <BFX-AT100_5.3.7_2023-0203-144713>, <BFX-AT100_5.3.7_2023-0203-142704>
Author: Sam Yang <sam.cy.yang@mail.foxconn.com>
AuthorDate: Thu Feb 2 15:47:12 2023 +0800
Commit: Sam Yang <sam.cy.yang@mail.foxconn.com>
CommitDate: Thu Feb 2 15:47:12 2023 +0800

    sepolicy: control volume in setupwraith_app

Issue History

Date Modified Username Field Change
2023-01-30 18:34 (SW) Jason Ling New Issue
2023-01-30 18:34 (SW) Jason Ling Status new => assigned
2023-01-30 18:34 (SW) Jason Ling Assigned To => (ALTech) JunGyu Kim
2023-01-30 18:34 (SW) Jason Ling Issue Monitored: (ALTech) Younkwang Jung
2023-01-30 18:34 (SW) Jason Ling Issue Monitored: (SW) Kerwin Chen
2023-01-30 18:34 (SW) Jason Ling Issue Monitored: (SW) Sam Yang
2023-01-31 09:02 (ALTech) JunGyu Kim Note Added: 0011896
2023-01-31 12:40 (ALTech) JunGyu Kim Note Added: 0011911
2023-02-01 11:56 (SW) Jason Ling Note Added: 0011939
2023-02-01 14:14 (ALTech) JunGyu Kim Note Added: 0011946
2023-02-01 15:12 (SW) Jason Ling Note Added: 0011956
2023-02-02 13:23 (ALTech) Sangmin Choi Note Added: 0011981
2023-02-02 14:44 (ALTech) Sangmin Choi Assigned To (ALTech) JunGyu Kim => (SW) Jason Ling
2023-02-02 15:30 (SW) Jason Ling Assigned To (SW) Jason Ling => (SW) Sam Yang
2023-02-02 15:30 (SW) Jason Ling Note Added: 0011983
2023-02-07 16:24 (SW) Jason Ling Status assigned => resolved
2023-02-07 16:24 (SW) Jason Ling Resolution open => fixed
2023-02-07 16:24 (SW) Jason Ling Note Added: 0012045
2023-02-07 16:25 (SW) Jason Ling Status resolved => closed